Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reporting to images #61

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

reporting to images #61

wants to merge 21 commits into from

Conversation

ThetaSinner
Copy link
Member

This is the big one, finally starting what we set out to do! :-)

@Sub-Xaero
Copy link
Contributor

This is the big one, finally starting what we set out to do! :-)

I'm so excited!!!!

@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@EphyraSoftware EphyraSoftware deleted a comment Oct 27, 2018
@Sub-Xaero
Copy link
Contributor

Still no idea why codacy is doing this. Its bloody annoying.

@ThetaSinner
Copy link
Member Author

ThetaSinner commented Oct 29, 2018

Lol, shtaaap deleting ma codacy-bot comments ;P

I assume it's a GH bug because codacy can't do that on your behalf...

@codecov-io
Copy link

codecov-io commented Oct 30, 2018

Codecov Report

Merging #61 into master will decrease coverage by 9.82%.
The diff coverage is 2.59%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #61      +/-   ##
============================================
- Coverage      43.4%   33.57%   -9.83%     
- Complexity       61       63       +2     
============================================
  Files            66       76      +10     
  Lines           728      959     +231     
  Branches        105      123      +18     
============================================
+ Hits            316      322       +6     
- Misses          371      596     +225     
  Partials         41       41
Impacted Files Coverage Δ Complexity Δ
...a/acropolis/service/config/ServiceConfiguration.kt 100% <ø> (ø) 0 <0> (ø) ⬇️
...tence/api/persistence/GraphicalAssetPersistence.kt 60% <0%> (-40%) 3 <0> (ø)
...ra/acropolis/service/impl/GraphicalAssetService.kt 75% <0%> (-10.72%) 0 <0> (ø)
...lin/org/ephyra/acropolis/report/api/model/Graph.kt 0% <0%> (ø) 0 <0> (?)
...lin/org/ephyra/acropolis/shell/BootstrapCommand.kt 0% <0%> (ø) 0 <0> (?)
...ra/acropolis/report/impl/render/DiagramRenderer.kt 0% <0%> (ø) 0 <0> (?)
...ephyra/acropolis/report/impl/render/CardBuilder.kt 0% <0%> (ø) 0 <0> (?)
.../org/ephyra/acropolis/report/impl/render/Size2D.kt 0% <0%> (ø) 0 <0> (?)
.../ephyra/acropolis/report/impl/render/Position2D.kt 0% <0%> (ø) 0 <0> (?)
...yra/acropolis/report/config/ReportConfiguration.kt 100% <100%> (ø) 1 <1> (?)
... and 13 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ffdb2ae...3406096. Read the comment docs.

@Sub-Xaero
Copy link
Contributor

Sub-Xaero commented Oct 31, 2018

Lol, shtaaap deleting ma codacy-bot comments ;P

I assume it's a GH bug because codacy can't do that on your behalf...

😆 Well, I've just blatted the Comment integration and it hasnt bitched about your latest commits, despite there being a missing comment in 548989c, so it hopefully is fixed. .

No comments at all is better than 200 deleted ones 🤷‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants